Skip to content

security: add bandit + semgrep scans and OpenSSF badge-ready policy docs#7

Merged
rlorenzo merged 1 commit intomainfrom
security-scanner
Apr 24, 2026
Merged

security: add bandit + semgrep scans and OpenSSF badge-ready policy docs#7
rlorenzo merged 1 commit intomainfrom
security-scanner

Conversation

@rlorenzo
Copy link
Copy Markdown
Owner

  • bandit runs in pre-commit at medium+ severity; -ll hides stylistic low-severity findings while still blocking on real risks.
  • semgrep runs in a new CI job with p/python, p/security-audit, and p/owasp-top-ten rule packs.
  • Single false-positive urlopen finding in updater.py (URL is a hardcoded https constant) suppressed inline with a justification comment.
  • CI workflow declares least-privilege contents: read, resolving five CodeQL "workflow missing permissions" alerts.
  • Adds SECURITY.md (private reporting via GitHub Security Advisories, 14-day response commitment, explicit in/out-of-scope), CONTRIBUTING.md (contribution, testing, and code-style policy), and CHANGELOG.md.
  • README cross-links the new policy docs, fixes a missing code-block language tag, and is rewrapped at 80 chars for cleaner diffs.

- bandit runs in pre-commit at medium+ severity; `-ll` hides stylistic
  low-severity findings while still blocking on real risks.
- semgrep runs in a new CI job with `p/python`, `p/security-audit`, and
  `p/owasp-top-ten` rule packs.
- Single false-positive urlopen finding in updater.py (URL is a hardcoded
  https constant) suppressed inline with a justification comment.
- CI workflow declares least-privilege `contents: read`, resolving five
  CodeQL "workflow missing permissions" alerts.
- Adds SECURITY.md (private reporting via GitHub Security Advisories,
  14-day response commitment, explicit in/out-of-scope), CONTRIBUTING.md
  (contribution, testing, and code-style policy), and CHANGELOG.md.
- README cross-links the new policy docs, fixes a missing code-block
  language tag, and is rewrapped at 80 chars for cleaner diffs.
@rlorenzo rlorenzo merged commit 255f827 into main Apr 24, 2026
9 checks passed
@rlorenzo rlorenzo deleted the security-scanner branch April 24, 2026 21:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant